Skip to content

Implemented a “Show More / Show Less” feature for skill suggestion chips on the homepage.#288

Open
Pranathi-Kunjeti wants to merge 3 commits into
komalharshita:mainfrom
Pranathi-Kunjeti:feat/limit-skill-suggestions
Open

Implemented a “Show More / Show Less” feature for skill suggestion chips on the homepage.#288
Pranathi-Kunjeti wants to merge 3 commits into
komalharshita:mainfrom
Pranathi-Kunjeti:feat/limit-skill-suggestions

Conversation

@Pranathi-Kunjeti
Copy link
Copy Markdown

Summary [required]

This PR improves the skill suggestion section on the homepage by limiting the number of visible quick-select skill chips initially and adding a “Show More / Show Less” toggle button.

The update reduces UI clutter on smaller screens and improves overall readability and responsiveness without affecting the existing skill selection functionality

Related Issue [required]

Closes #195

Type of Change [required]

  • Bug fix — resolves a broken behaviour
  • Feature — adds new functionality
  • Data — adds new projects to data/projects.json
  • Documentation — updates docs, README, or code comments only
  • Style — CSS or visual changes only, no logic change
  • Refactor — restructures code without changing behaviour
  • Test — adds or updates tests

What Was Changed [required]

File Change made
templates/index.html Added Show More / Show Less toggle button and grouped extra skill chips
static/style.css Added styles for hidden extra skills and toggle button
static/script.js Added toggle functionality for expanding/collapsing extra skill suggestions

How to Test This PR [required]

  1. Clone this branch: git checkout feat/limit-visible-skill-suggestions
  2. Install dependencies: pip install -r requirements.txt
  3. Run the app: python app.py
  4. Open http://127.0.0.1:5000
  5. Run the tests: python tests/test_basic.py

Expected test output:

30 passed, 0 failed out of 30 tests

Test Results [required]

29 passed, 1 failed out of 30 tests
 Failing test: FAIL test_health_check: test_health_check() missing 1 required positional argument: 'client'

Screenshots (if UI change)

Before :
Screenshot 2026-05-18 163343

After:
Screenshot 2026-05-18 163409

Screenshot 2026-05-18 163418

Self-Review Checklist [required]

  • I have read CONTRIBUTING.md and followed all guidelines
  • My branch name follows the convention: feat/, fix/, docs/, data/, style/, test/
  • I have run python tests/test_basic.py and all 29 tests pass
  • I have run flake8 . locally and there are no errors
  • I have not introduced any print() or console.log() debug statements
  • Every new function I wrote has a docstring
  • I have not modified files outside the scope of the linked issue
  • If I changed the UI, I tested it at 375px (mobile) and 1280px (desktop)
  • If I added a project to the dataset, it has all required JSON fields

Notes for Reviewer

One existing test (test_health_check) is currently failing due to a missing required positional argument (client). This issue is unrelated to the changes introduced in this PR.

@vercel
Copy link
Copy Markdown

vercel Bot commented May 18, 2026

@Pranathi-Kunjeti is attempting to deploy a commit to the komalsony234-1530's projects Team on Vercel.

A member of the Team first needs to authorize it.

Copy link
Copy Markdown

@github-actions github-actions Bot left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thank you for submitting your first pull request to DevPath.

Before review:

  • Complete the PR template fully
  • Ensure all tests pass
  • Link your PR to an issue
  • Keep changes scoped to the issue

A maintainer will review your contribution soon.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[Feature]: Limit visible skill suggestions with “Show More” option

1 participant